Skip to content

Fix incorrect port mapping of 2368 and update to latest docker image version#1713

Closed
hong823 wants to merge 2 commits intodocker-library:masterfrom
hong823:master
Closed

Fix incorrect port mapping of 2368 and update to latest docker image version#1713
hong823 wants to merge 2 commits intodocker-library:masterfrom
hong823:master

Conversation

@hong823
Copy link

@hong823 hong823 commented May 3, 2020

Ghost's port is displayed as 8080 which is incorrect, port 2368 should be the correct port as there are URLs on the page that is redirecting to port 2368.

Example:

http://localhost:2368/author/ghost/

@hong823 hong823 changed the title Fix incorrect port mapping of 2368 Fix incorrect port mapping of 2368 and update to latest docker image version May 3, 2020
@tianon
Copy link
Member

tianon commented May 4, 2020

It's not "incorrect" so much as "intentionally using a different host port than the port inside the container" which is why the line in the YAML has two values (the first being the "outside port" and the second being the "inside port" which is the port Ghost/Node is listening on).

What we should probably do instead is add an example url environment variable (which is a commonly requested addition) pointing to the exposed port.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants